Skip to content

Assign priority to issues from Zulip#2337

Merged
apiraino merged 1 commit intorust-lang:masterfrom
apiraino:assign-priority-from-zulip
Mar 10, 2026
Merged

Assign priority to issues from Zulip#2337
apiraino merged 1 commit intorust-lang:masterfrom
apiraino:assign-priority-from-zulip

Conversation

@apiraino
Copy link
Contributor

@apiraino apiraino commented Mar 6, 2026

Triagebot learns how to post a comment on GitHub to assign priority to an issue that is marked as regression. The command is sent from the Zulip Rust lang chat instance. The syntax is:

@triagebot prio #issue <priority>

example @triagebot prio #123456 high

The comment on GitHub is my usual comment on issue priority:

Assigning P-high (discussion on [Zulip](<zulip-link>)).

This handler checks that:

  • issue number is really an issue (prevents assigning priority to PRs)
  • the priority value is allowed

r? @Urgau

Thanks for a review!

@apiraino apiraino force-pushed the assign-priority-from-zulip branch from 79a594b to 4f9726f Compare March 6, 2026 15:18
@apiraino apiraino requested a review from Kobzol March 6, 2026 15:19
@apiraino apiraino enabled auto-merge March 6, 2026 15:28
Copy link
Member

@Kobzol Kobzol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Left one comment.

View changes since this review

@apiraino apiraino force-pushed the assign-priority-from-zulip branch from 4f9726f to a7f7eb2 Compare March 9, 2026 16:43
@apiraino apiraino requested a review from Kobzol March 9, 2026 16:44
@apiraino apiraino added this pull request to the merge queue Mar 10, 2026
Merged via the queue into rust-lang:master with commit 27ba888 Mar 10, 2026
3 checks passed
@apiraino apiraino deleted the assign-priority-from-zulip branch March 10, 2026 07:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants